Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Track panic mode in fingerprint #5458

Merged
merged 1 commit into from
May 2, 2018
Merged

Conversation

alexcrichton
Copy link
Member

Ensure that if we've previously compiled a crate with panic=abort and we later
need it for panic=unwind we correctly recompile it.

Closes #5445

@rust-highfive
Copy link

r? @matklad

(rust_highfive has picked a reviewer for you, use r? to override)

@matklad
Copy link
Member

matklad commented May 2, 2018

@bors r+

@bors
Copy link
Contributor

bors commented May 2, 2018

📌 Commit 0dd9c74 has been approved by matklad

Ensure that if we've previously compiled a crate with panic=abort and we later
need it for panic=unwind we correctly recompile it.

Closes rust-lang#5445
@matklad
Copy link
Member

matklad commented May 2, 2018

@bors r+

@bors
Copy link
Contributor

bors commented May 2, 2018

📌 Commit 7f44648 has been approved by matklad

@bors
Copy link
Contributor

bors commented May 2, 2018

⌛ Testing commit 7f44648 with merge 31b38fe...

bors added a commit that referenced this pull request May 2, 2018
Track panic mode in fingerprint

Ensure that if we've previously compiled a crate with panic=abort and we later
need it for panic=unwind we correctly recompile it.

Closes #5445
@bors
Copy link
Contributor

bors commented May 2, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing 31b38fe to master...

@bors bors merged commit 7f44648 into rust-lang:master May 2, 2018
@alexcrichton alexcrichton deleted the track-panic branch May 10, 2018 19:17
@ehuss ehuss added this to the 1.27.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

panic not always tracked in built dependencies
5 participants